-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Core: Add multiple attribute to email method. #2074
base: master
Are you sure you want to change the base?
Core: Add multiple attribute to email method. #2074
Conversation
This issue/proposal has been automatically marked as idle and stale because it hasn't had any recent activity. It will be automtically closed if no further activity occurs. If you think this is wrong, or the problem still persists, just pop a reply in the comments and one of the maintainers will (try!) to follow up. |
I think this PR still has merit. |
Hi @RobJohnston, Sorry for taking so long to review this PR. I only have one concern about this implementation. That What I would like to suggest is, we add support for passing an object to $("form").validate({
rules: {
field: {
email: true | false | {
multiple: true | false
}
}
}
}); The |
Hi @Arkni . It's now my time to apologize to you for the delay! I've been away from front-end development for some time. I didn't quite understand your comment when I first read it and not sure if I do now. From the W3, the multiple attribute is what they call a boolean attribute, but that doesn't mean that it can be set to "true" or "false". It must be multiple: true | false |
Using @Sven-Mad's method. Fixes #1722.